From 36ac3bcd564975302e6e6df556b854687c0d1f93 Mon Sep 17 00:00:00 2001 From: Ian Jackson Date: Fri, 28 Jan 2011 18:39:09 +0000 Subject: [PATCH] libxl: prevent creation of domains with duplicate names libxl_domain_rename is where domain names are assigned. Therefore this is where we check that no two domains have the same name. As a special exception, domains whose names are "" are not considered to clash. We also take special care not to mind if we try to rename a domain to the name it already has. Signed-off-by: Ian Jackson Acked-by: Stefano Stabellini Committed-by: Ian Jackson --- tools/libxl/libxl.c | 22 ++++++++++++++++++++++ tools/libxl/libxl_create.c | 1 + 2 files changed, 23 insertions(+) diff --git a/tools/libxl/libxl.c b/tools/libxl/libxl.c index 9658d6714e..2d2b884baf 100644 --- a/tools/libxl/libxl.c +++ b/tools/libxl/libxl.c @@ -141,6 +141,28 @@ int libxl_domain_rename(libxl_ctx *ctx, uint32_t domid, } } + if (new_name[0]) { + /* nonempty names must be unique */ + uint32_t domid_e; + rc = libxl_name_to_domid(ctx, new_name, &domid_e); + if (rc == ERROR_INVAL) { + /* no such domain, good */ + } else if (rc != 0) { + LIBXL__LOG(ctx, LIBXL__LOG_ERROR, "unexpected error" + "checking for existing domain"); + goto x_rc; + } else if (domid_e == domid) { + /* domain already has this name, ok (but we do still + * need the rest of the code as we may need to check + * old_name, for example). */ + } else { + LIBXL__LOG(ctx, LIBXL__LOG_ERROR, "domain with name \"%s\"" + " already exists.", new_name); + rc = ERROR_INVAL; + goto x_rc; + } + } + if (old_name) { got_old_name = xs_read(ctx->xsh, trans, name_path, &got_old_len); if (!got_old_name) { diff --git a/tools/libxl/libxl_create.c b/tools/libxl/libxl_create.c index 1879de048d..dc2621468e 100644 --- a/tools/libxl/libxl_create.c +++ b/tools/libxl/libxl_create.c @@ -351,6 +351,7 @@ int libxl__domain_make(libxl_ctx *ctx, libxl_domain_create_info *info, retry_transaction: t = xs_transaction_start(ctx->xsh); + xs_rm(ctx->xsh, t, dom_path); xs_mkdir(ctx->xsh, t, dom_path); xs_set_permissions(ctx->xsh, t, dom_path, roperm, ARRAY_SIZE(roperm)); -- 2.30.2